Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement flux debug kustomization command #5117

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

stefanprodan
Copy link
Member

@stefanprodan stefanprodan commented Dec 13, 2024

This PR implements the flux debug ks command as specified in #5109

Followup: #5106
Closes: #5109

@stefanprodan stefanprodan added area/kustomization Kustomization related issues and pull requests area/ux In pursuit of a delightful user experience labels Dec 13, 2024
}
testEnv.CreateObjectFile(objectFile, tmpl, t)

cases := []struct {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add 2 more tests: one for both flags set, and one for neither flags set?

cmd/flux/debug_kustomization.go Show resolved Hide resolved
cmd/flux/debug_kustomization.go Show resolved Hide resolved
cmd/flux/debug_kustomization.go Show resolved Hide resolved
Copy link
Contributor

@darkowlzz darkowlzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Left a comment about similar changes to debug helmrelease, which can be addresses here or separately.

Comment on lines +69 to +71
if (!debugKustomizationArgs.showStatus && !debugKustomizationArgs.showVars) ||
(debugKustomizationArgs.showStatus && debugKustomizationArgs.showVars) {
return fmt.Errorf("either --show-status or --show-vars must be set")
Copy link
Contributor

@darkowlzz darkowlzz Dec 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed before, we need to do this for debug helmrelease too. Maybe as part of this change or a separate PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kustomization Kustomization related issues and pull requests area/ux In pursuit of a delightful user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Troubleshoot helpers for Kustomization reconcile failures
3 participants